Skip to content

Comments

WA-NEW-009: Replace deprecated update_attributes in production code#633

Open
kitcommerce wants to merge 1 commit intonextfrom
wa-new-009-replace-update-attributes
Open

WA-NEW-009: Replace deprecated update_attributes in production code#633
kitcommerce wants to merge 1 commit intonextfrom
wa-new-009-replace-update-attributes

Conversation

@kitcommerce
Copy link

Summary

Replace all deprecated update_attributes / update_attributes! calls with update / update! across core production code (core/app/**, core/lib/**). This is required for Rails 7 where update_attributes is removed.

Also includes the WA-NEW-008 password reset fix (error-copying updated for Rails 7 compatibility).

Closes #626

Client impact

None expected. update / update! are drop-in replacements with identical behavior. No public API changes.

Verify

bundle exec ruby -Icore/test core/test/models/workarea/user/password_reset_test.rb

Files changed

17 files across core models, services, workers, seeds, and lib.

@kitcommerce
Copy link
Author

Dispatcher Build Gate Summary (local)

  • rubocop (diff-only): PASS (0 offenses)
  • brakeman: skipped (per repo build gate config; disabled)
  • tests (affected engines): PASS
    • core: PASS

Note: test output still prints BSON Symbol deprecation warning (expected to be addressed by WA-NEW-010 / PR #635).

@kitcommerce kitcommerce added review:architecture-pending Review in progress review:simplicity-pending Review in progress review:security-pending Review in progress review:rails-conventions-pending Rails conventions review in progress and removed review:architecture-pending Review in progress review:security-pending Review in progress labels Feb 22, 2026
@kitcommerce
Copy link
Author

⚠️ Review orchestration paused: hit an LLM API rate limit while spawning Wave 1 reviewer agents for this PR.

No action needed from the PR author; I’ll retry dispatch once limits clear.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

gate:build-passed Build gate passed review:rails-conventions-pending Rails conventions review in progress review:simplicity-pending Review in progress

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant